Repository navigation
fix(devex): read depot usage one day at a time in the weekly digest - #131885
Conversation
|
😎 Merged successfully - details. |
🤖 CI report🚨 Trunk lane — universal laneThis PR is assigned to the universal lane. It cannot merge in parallel with other PRs, so it can take longer to merge. Ask dev-ex if you think this is wrong. |
|
Risk: No findings This PR changes the weekly CI digest to read Depot usage one UTC day at a time (working around Depot GetUsage 500s on 7-day ranges), caches day reads so the contract range reuses the two weeks already read, adds a 20-minute time budget after which the digest posts without spend, and raises the job timeout from 12 to 30 minutes. I traced the Depot response parsing, the day cache for races, the time-budget overshoot math, and the secrets/log handling, and found no security issue: all inputs are validated numeric aggregates from authenticated APIs, spends run serially so no double-read race exists, and error paths degrade to posting without the section rather than leaking data. Sentinel reviewed |
|
|
PR overviewNo security issues were found in this pull request. Reviews
|
![stamphog[bot]](https://avatars.githubusercontent.com/in/3000332?s=60&v=4){kind=small}
There was a problem hiding this comment.
Approved.
The CI changes have owning-team authorship, and trusted context reports a current-head automated review with no findings, though its review state is unspecified. No substantive review concerns or other refusal grounds were found.
- Author wrote 26% of the modified lines and has 148 merged PRs in these paths (familiarity MODERATE).
- PostHog Review (Standard) reviewed the current head: no findings.
Gate mechanics and policy version
| Gate | Result | |
|---|---|---|
| prerequisites | ✓ | all clear |
| deny-list | ✓ | no deny categories matched |
| size | ✓ | 56L, 2F substantive, 110L/3F incl. docs/generated/snapshots — within ceiling |
| tier | ✓ | T1-agent / T1c-medium (110L, 3F, single-area, fix) |
| stamphog 2.7.0 | .stamphog/policy.yml @ 52672cf · reviewed head 52672cf |
Deploy status
This table shows the |
Problem
Changes
How did you test this code?
dry_rundispatch after merge confirms it.Test rationale: The contract test now fails if a Depot request spans more than one day. New cases catch a contract total that counts pre-contract days, and a time-budget stop that drops all spend.
Release status
Docs update
🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Agent: Claude Code, Opus 5.5 (claude-opus-5-5)
Skills:
/authoring-ci-workflows,/writing-tests,/writing-code-comments,/writing-pr-descriptions,/code-review,/simplifyMore context for AI agents
spend, which lets the contract range reuse both weeks.